refactor(plugin)!: rename /stocks:reports to /stocks:list - #337
Merged
Conversation
BREAKING CHANGE: `/stocks:reports` is now `/stocks:list`. `report` and `reports` differed by one letter and by nothing else visible at the prompt — singular takes a symbol and writes one report, plural takes none and lists the whole covered set. That is a coin-flip every time, and it only became a real hazard in v0.30.0 when `/stocks:stocks` became `/stocks:report` and put the two names next to each other. `list` rather than `index`, which is the other obvious candidate: in a plugin about equities, `/stocks:index` reads as a market index. `list` cannot. The CLI needs no change — `moshcode stocks list` has been an alias for `reports` since the verb existed, so this makes the plugin agree with the CLI rather than diverge from it. Both plugins go to 0.4.0. crypto is bumped too, and not just for symmetry: its `/crypto:help` names the stocks command set, so its content changed as well and an install would otherwise keep printing a command that no longer exists. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
ThreatCrush Security Scan91 finding(s) HIGH/CRITICAL: 2 | MEDIUM: 41 | LOW: 48
…and 41 more. Full results in the Security tab. Snippets are redacted; ThreatCrush never prints matched credential material. |
Merged
ralyodio
added a commit
that referenced
this pull request
Aug 9, 2026
Bump to v0.31.0, releasing the /stocks:reports → /stocks:list rename (#337). Breaking, in the plugins rather than the CLI, and a direct consequence of v0.30.0: renaming /stocks:stocks to /stocks:report put `report` and `reports` next to each other, one letter apart, which is a coin-flip at the prompt. `moshcode stocks list` already worked, so the CLI is untouched — this makes the plugin agree with it. Both plugins go to 0.4.0. crypto changed too: its /crypto:help names the stocks command set, so an un-bumped install would keep printing a command that no longer exists. Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
BREAKING CHANGE:
/stocks:reportsis now/stocks:list.reportandreportsdiffered by one letter and by nothing else visible at the prompt — singular takes a symbol and writes one report, plural takes none and lists the whole covered set. That's a coin-flip every time, and it only became a real hazard in v0.30.0, when/stocks:stocksbecame/stocks:reportand put the two names right next to each other.Why
listand notindexindexwas the other obvious candidate and is the word the repo already uses in prose ("the stored index"). But in a plugin about equities,/stocks:indexreads as a market index — someone would reasonably expect S&P 500 data.listcan't be misread that way.The CLI needs no change
moshcode stocks listhas been an alias forreportssince the verb existed:So this makes the plugin agree with the CLI rather than diverge from it.
Versions
Both plugins go to 0.4.0. crypto is bumped too, and not just for symmetry: its
/crypto:helpnames the stocks command set, so its content changed as well — without the bump an existing install would keep printing a command that no longer exists.Verification
moshcode plugin listshows/stocks:listin place of/stocks:reports.🤖 Generated with Claude Code